Skip to content

ci: pin GitHub Actions to commit SHAs#5

Merged
motsc merged 1 commit into
mainfrom
ci/pin-actions-to-shas
Jun 22, 2026
Merged

ci: pin GitHub Actions to commit SHAs#5
motsc merged 1 commit into
mainfrom
ci/pin-actions-to-shas

Conversation

@motsc

@motsc motsc commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Pins the actions in both workflows to the commit SHA each tag points at right now, version left as a trailing comment so Dependabot can still bump them.

Action Tag SHA
actions/checkout v6 df4cb1c069e1874edd31b4311f1884172cec0e10
oven-sh/setup-bun v2 0c5077e51419868618aeaa5fe8019c62421857d6
actions/cache v5 27d5ce7f107fe9357f9df03efb73ab90386fccae

the only one that really matters is oven-sh/setup-bun, it's third-party so a repointed tag would run in CI. the actions/* ones are github-owned, pinned just for consistency. blast radius is small anyway, CI runs with a read-only token and there are no actions secrets in the repo.

heads up, this repo is published from the ai monorepo so the next sync will revert these unless the same pin lands upstream. not enforcing sha pinning at the repo level for that reason, the branch protection + required checks cover main either way.

Replace mutable version tags (@v6, @v2, @v5) with the commit SHA each
tag currently resolves to, keeping the version as a trailing comment so
Dependabot can still propose bumps. Pinning the third-party action
(oven-sh/setup-bun) closes a supply-chain vector where a repointed tag
would execute in CI; the first-party actions/* are pinned for
consistency.

- actions/checkout   v6 -> df4cb1c069e1874edd31b4311f1884172cec0e10
- oven-sh/setup-bun  v2 -> 0c5077e51419868618aeaa5fe8019c62421857d6
- actions/cache      v5 -> 27d5ce7f107fe9357f9df03efb73ab90386fccae
@motsc motsc requested review from danny-avila June 22, 2026 14:08
@motsc motsc merged commit dbc7419 into main Jun 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants